Skip to content

Disable unit tests for flakyness in CircleCI - #45133

Closed
cipolleschi wants to merge 1 commit into
react:mainfrom
cipolleschi:export-D58948614
Closed

Disable unit tests for flakyness in CircleCI#45133
cipolleschi wants to merge 1 commit into
react:mainfrom
cipolleschi:export-D58948614

Conversation

@cipolleschi

Copy link
Copy Markdown
Contributor

Summary:
Recently, CircleCI has failed pretty often due to these tests.
The reason is that we migrated these jobs to M1 machines, as circleCI is deprecating intel ones, and on these machines the simulators tend to freeze. Hence, every often we get stuck when running the tests. :(

We are going to move to GHA, so that should not be a big issue.

Changelog:

[Internal] - Disable unit tests in CircleCI to improve CI robustness

Differential Revision: D58948614

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Jun 24, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D58948614

@analysis-bot

analysis-bot commented Jun 24, 2024

Copy link
Copy Markdown
Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 20,316,960 +16
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 23,513,639 -1
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: c0b5e2e
Branch: main

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D58948614

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D58948614

Summary:
Pull Request resolved: #45133

Recently, CircleCI has failed pretty often due to these tests.
The reason is that we migrated these jobs to M1 machines, as circleCI is deprecating intel ones, and on these machines the simulators tend to freeze. Hence, every often we get stuck when running the tests. :(

We are going to move to GHA, so that should not be a big issue.

## Changelog:
[Internal] - Disable unit tests in CircleCI to improve CI robustness

Reviewed By: cortinico

Differential Revision: D58948614
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D58948614

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in 2adb389.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jun 27, 2024
@github-actions

Copy link
Copy Markdown

This pull request was successfully merged by @cipolleschi in 2adb389.

When will my fix make it into a release? | How to file a pick request?

@kraenhansen

Copy link
Copy Markdown
Contributor

@cipolleschi the description of this PR details that it disables the unit tests on Circle CI and mentions that it's not a big issue since they're ran on GHA.

However, this PR also disables tests on GHA as well: 2adb389#diff-12d2c3c126fcc84106dce4b6d2be2220f018661b111a5382ad6effc1047f66aeL435 and I suspect that wasn't intentional.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants